Skip to content
This repository has been archived by the owner on Jan 18, 2024. It is now read-only.

update SvelteKit example (#166) #175

Closed
wants to merge 5 commits into from
Closed

update SvelteKit example (#166) #175

wants to merge 5 commits into from

Conversation

jaads
Copy link

@jaads jaads commented Nov 23, 2022

Fixes #166

@bwklein
Copy link

bwklein commented Nov 27, 2022

@jaads Thank you for doing this, I was about to start a PR and saw that you just did it and are saving me a bunch of time!

@jaads
Copy link
Author

jaads commented Nov 27, 2022

No problem! Very glad to hear that!! 😊😊

@KoljaL
Copy link

KoljaL commented Dec 6, 2022

Hi @jaads
thanks a lot for updating the SvelteKit example.
I used your fork and got an error: AxiosError: connect ECONNREFUSED ::1:8055

My solution, mentioned on the axios repo, is to replace the localhostwith 127.0.0.1 here:
https://github.com/jaads/directus-examples/blob/main/sveltekit/.env.example#L1

Greetings from DO

@jaads
Copy link
Author

jaads commented Dec 6, 2022

Alright, it was already done here #177 . I just merged the upstream into my fork, so it should work now also with newer versions of node :)

@alexvdvalk
Copy link

I just made another pull request as Sveltekit v1 was just released. Which ever gets merged, it should be an easy upgrade from there.

@jaads
Copy link
Author

jaads commented Jan 2, 2023

Since this PR is kind of outdated due to #178 I'll just close this one

@jaads jaads closed this Jan 2, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Sveltekit example out of date.
5 participants